Skip to content

test: remove network dependencies and extend coverage#13

Merged
spiegel-im-spiegel merged 1 commit into
mainfrom
fix/tests-remove-network-dependency
May 9, 2026
Merged

test: remove network dependencies and extend coverage#13
spiegel-im-spiegel merged 1 commit into
mainfrom
fix/tests-remove-network-dependency

Conversation

@spiegel-im-spiegel
Copy link
Copy Markdown
Member

@spiegel-im-spiegel spiegel-im-spiegel commented May 9, 2026

Summary

  • replace external-network-dependent tests with deterministic local tests (httptest and mocked transport)
  • add POST-path tests for success/error/nil URL scenarios
  • add request option header tests for WithRequestHeaderAdd and WithRequestHeaderSet
  • add internal response lifecycle tests for DumpBodyAndClose behavior
  • add tests for URL trim behavior, deprecated method delegation, and 3xx status handling
  • update copyright years in touched files

Validation

  • task test

@spiegel-im-spiegel spiegel-im-spiegel merged commit 01859f9 into main May 9, 2026
4 checks passed
@spiegel-im-spiegel spiegel-im-spiegel deleted the fix/tests-remove-network-dependency branch May 9, 2026 13:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@spiegel-im-spiegel